Skip to content

Define (and use) custom component "RestRef"#155

Merged
kthoms merged 4 commits into
operaton:mainfrom
andreibancioiu:rest-api-doc-link
Jun 20, 2026
Merged

Define (and use) custom component "RestRef"#155
kthoms merged 4 commits into
operaton:mainfrom
andreibancioiu:rest-api-doc-link

Conversation

@andreibancioiu

@andreibancioiu andreibancioiu commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • define the reusable RestRef MDX component for links into the published REST API reference
  • support tag/operation, drop the legacy tag/page
  • render external REST API links with target="_blank" and rel="noreferrer"
  • migrate all remaining old lowercase restref snippets in the docs to RestRef, preserving tag/operation deep links instead of flattening them to the REST overview
  • polish the touched REST API wording in the REST specification and external task docs

Verification

  • rg -n --case-sensitive 'restref ' docs build finds no remaining old lowercase restref snippets
  • fetched https://docs.operaton.org/reference/latest/rest-api/operaton-rest-api.json and confirmed every migrated tag/operation id exists in the current REST API spec
  • spot-checked generated REST API URLs for Deployment, External Task, and Task operations with curl; all returned HTTP 200
  • git diff --check
  • npm run build succeeds; this branch still reports the known pre-existing Docusaurus broken link/anchor warnings handled separately in Fix docs link checks and enforce quality gates #115

Notes

### Rest API

See the restref text="REST API documentation" tag="External-Task for how the API operations can be accessed via HTTP.
See the <RestRef text="REST API documentation" section="External-Task" /> for how the API operations can be accessed via HTTP.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All right to use the name "section" instead of "tag"?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may look rendered as a section, but "tag" is more precise in regard to the Open API spec
https://swagger.io/docs/specification/v3_0/grouping-operations-with-tags/

@andreibancioiu andreibancioiu Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed to use "tag" and "operation".

@hauptmedia, I hope you don't mind the switch from "page" to "operation". In the PR description, I see "page" was called legacy.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for me fine

Comment thread src/components/RestRef.jsx Outdated
Comment thread src/theme/MDXComponents.js
Comment thread docusaurus.config.ts Outdated
Comment thread docs/documentation/user-guide/process-engine/history/history-cleanup.md Outdated
@kthoms

kthoms commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Thank you @andreibancioiu for proposing this component. It will be helpful. Let's polish and use it then.

Comment thread docs/documentation/user-guide/process-engine/external-tasks.md Outdated
Comment thread src/theme/MDXComponents.js
@andreibancioiu andreibancioiu requested a review from kthoms June 18, 2026 18:49
@kthoms

kthoms commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Looks good to me. One last formal thing: Please just copy and paste this statement as first-time-contributor

I confirm that my contribution and following ones comply with the Apache License 2.0 and the Code of Conduct.

@andreibancioiu

Copy link
Copy Markdown
Contributor Author

I confirm that my contribution and following ones comply with the Apache License 2.0 and the Code of Conduct.

@kthoms kthoms merged commit e42eab3 into operaton:main Jun 20, 2026
@kthoms

kthoms commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Thank you a lot for this component, @andreibancioiu !

@andreibancioiu andreibancioiu deleted the rest-api-doc-link branch June 22, 2026 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants